Skip to content

chore: cargo fmt#563

Merged
mgeisler merged 3 commits intomgeisler:masterfrom
waywardmonkeys:fix-fmt
Mar 3, 2025
Merged

chore: cargo fmt#563
mgeisler merged 3 commits intomgeisler:masterfrom
waywardmonkeys:fix-fmt

Conversation

@waywardmonkeys
Copy link
Copy Markdown
Contributor

No description provided.

@waywardmonkeys
Copy link
Copy Markdown
Contributor Author

I guess several other things need fixing for the new CI check to pass ... what do you think about if it should be done or not?

@mgeisler
Copy link
Copy Markdown
Owner

mgeisler commented Oct 1, 2024

I guess several other things need fixing for the new CI check to pass ... what do you think about if it should be done or not?

I would prefer to not enforcing this in CI — the library doesn't get a lot of contributions, but when it does get a patch, there is no need to be super demanding on this. I don't feel this has been a problem in the past.

I like dprint and use it in other projects, so feel free to update the PR to keep the config file.

Uses the same check / action as in `smawk`.
@mgeisler mgeisler enabled auto-merge (squash) March 3, 2025 14:17
@mgeisler
Copy link
Copy Markdown
Owner

mgeisler commented Mar 3, 2025

Let's get this merged so I can make a new shiny release!

@mgeisler mgeisler merged commit 867bea5 into mgeisler:master Mar 3, 2025
23 of 26 checks passed
@github-actions github-actions bot mentioned this pull request Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants